messageTray: Do not add the tray unconditionally to ctrl-alt-tab

Commit 448517032e5de87 added the message tray unconditionally to
the Ctrl-Alt-Tab popup, but while this makes sense for a normal
session, we do not want it in the login screen.
Be a bit more careful where we make the tray available.

https://bugzilla.gnome.org/show_bug.cgi?id=685914

diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 982940abe5529fb8586ac5030f7b56017ee8222b..30ce5cc1c0ee045a283f5879bd63ab1d4d8b8a4f 100644 (file)
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -1514,7 +1514,7 @@ const MessageTray = new Lang.Class({
                 this._updateState();
             }));
 
-        Main.sessionMode.connect('updated', Lang.bind(this, this._updateState));
+        Main.sessionMode.connect('updated', Lang.bind(this, this._sessionUpdated));
 
         global.display.add_keybinding('toggle-message-tray',
                                       new Gio.Settings({ schema: SHELL_KEYBINDINGS_SCHEMA }),
@@ -1530,9 +1530,17 @@ const MessageTray = new Lang.Class({
         this._trayDwelling = false;
         this._trayDwellUserTime = 0;
 
-        Main.ctrlAltTabManager.addGroup(this._summary, _("Message Tray"), 'start-here-symbolic',
-                                        { focusCallback: Lang.bind(this, this.toggleAndNavigate),
-                                          sortGroup: CtrlAltTab.SortGroup.BOTTOM });
+        this._sessionUpdated();
+    },
+
+    _sessionUpdated: function() {
+        if (Main.sessionMode.isLocked || Main.sessionMode.isGreeter)
+            Main.ctrlAltTabManager.removeGroup(this._summary);
+        else
+            Main.ctrlAltTabManager.addGroup(this._summary, _("Message Tray"), 'start-here-symbolic',
+                                            { focusCallback: Lang.bind(this, this.toggleAndNavigate),
+                                              sortGroup: CtrlAltTab.SortGroup.BOTTOM });
+        this._updateState();
     },
 
     _checkTrayDwell: function(x, y) {